Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix negative strand CIGAR renderings on linear synteny view #3082

Merged
merged 3 commits into from
Jul 11, 2022

Conversation

cmdcolin
Copy link
Collaborator

Inspection of the DNA bases found that what was being rendered as the CIGAR (e.g. yellow and blue insertions/deletions) did not match up properly, so this fixes the issue

@github-actions github-actions bot added the needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) label Jul 11, 2022
@cmdcolin cmdcolin force-pushed the fix_neg_strand_synteny_cigar branch from fe3e185 to 7e3626b Compare July 11, 2022 01:22
@cmdcolin cmdcolin force-pushed the fix_neg_strand_synteny_cigar branch from 7e3626b to 03ab944 Compare July 11, 2022 01:23
@cmdcolin
Copy link
Collaborator Author

image of the fix after this branch is appliedScreenshot from 2022-07-10 19-24-22

@codecov
Copy link

codecov bot commented Jul 11, 2022

Codecov Report

Merging #3082 (c4da8a4) into main (edd9360) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

❗ Current head c4da8a4 differs from pull request most recent head c2ca98e. Consider uploading reports for the commit c2ca98e to get more accurate results

@@            Coverage Diff             @@
##             main    #3082      +/-   ##
==========================================
- Coverage   61.35%   61.34%   -0.02%     
==========================================
  Files         594      594              
  Lines       27277    27282       +5     
  Branches     6596     6601       +5     
==========================================
- Hits        16736    16735       -1     
- Misses      10243    10249       +6     
  Partials      298      298              
Impacted Files Coverage Δ
.../comparative-adapters/src/PAFAdapter/PAFAdapter.ts 64.61% <ø> (ø)
...tenyRenderer/components/LinearSyntenyRendering.tsx 53.38% <40.00%> (-0.45%) ⬇️
...lugins/alignments/src/BamAdapter/MismatchParser.ts 89.15% <100.00%> (ø)
...otplot-view/src/DotplotRenderer/DotplotRenderer.ts 77.95% <0.00%> (-1.58%) ⬇️
products/jbrowse-web/src/util.ts 27.27% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edd9360...c2ca98e. Read the comment docs.

@cmdcolin cmdcolin added bug Something isn't working and removed needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) labels Jul 11, 2022
@cmdcolin cmdcolin changed the title Fixes to negative strand CIGAR renderings on linear synteny view Fix negative strand CIGAR renderings on linear synteny view Jul 11, 2022
@cmdcolin
Copy link
Collaborator Author

has to use a special flag because we do not want to apply this to the read vs ref view, in that case the CIGAR is recorded in the reverse orientation already

@cmdcolin cmdcolin force-pushed the fix_neg_strand_synteny_cigar branch from 93d9c7d to c2ca98e Compare July 11, 2022 02:35
@cmdcolin cmdcolin merged commit d10927a into main Jul 11, 2022
@cmdcolin cmdcolin deleted the fix_neg_strand_synteny_cigar branch July 11, 2022 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant